-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create sample app #17
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think ok this sample app. I commented that I point of confusion.
-- original text --
アプリの方は問題なさそうだけど、気になる点についてコメントした
source 'https://rubygems.org' | ||
git_source(:github) { |repo| "https://github.com/#{repo}.git" } | ||
|
||
ruby '3.0.1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't agree to fix the ruby version in this sample app.
-- original text --
サンプルならいらない
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katsyoshi I agree to remove the ruby version specify from "Gemfile". But, a rails app needs "a ruby version", right?
(Gemfileからrubyのversionを消すのは構わないですが、何らかのRubyのversion指定なしにRailsって起動できましたっけ?)
@@ -0,0 +1,261 @@ | |||
PATH |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why committed this file which is ignored?
-- original text --
ignore しているのに入れるの?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rails appのGemfile.lockのなので入れたいです、git ignoreしているというのは、mini_paperclipのtoplevel dirにあるGemfile.lockのことで、sample/Gemfile.lock
は !
で無視しないようにしています。
require "rails" | ||
# Pick the frameworks you want: | ||
require "active_model/railtie" | ||
require "active_job/railtie" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nits]
I seem that didn't use active_job.
-- original text --
使ってないのでは?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
使ってない確かに (rails newのoptionは全然つけていなかったので)
A working sample is important for a user.